Skip to content

Parity CI hardening + FitResSummary checkpoint regression guard#5

Closed
iahncajigas wants to merge 15 commits intomainfrom
codex/parity-line-review-20260303
Closed

Parity CI hardening + FitResSummary checkpoint regression guard#5
iahncajigas wants to merge 15 commits intomainfrom
codex/parity-line-review-20260303

Conversation

@iahncajigas
Copy link
Copy Markdown
Contributor

@iahncajigas iahncajigas commented Mar 3, 2026

Summary

  • enforce non-LFS checkout across remaining CI workflows to prevent checkout failures when LFS quota is exhausted
  • add a focused regression test for FitResSummaryExamples notebook checkpoints
  • add tools/parity/build_line_review_sprint.py and generated parity/line_review_sprint.md to prioritize needs_review topics by lowest line-alignment ratio

Validation

  • pytest -q tests/test_fitressummary_notebook_checkpoint.py tests/test_notebook_numeric_checkpoints.py tests/test_notebook_assertions.py
  • python tools/parity/build_line_review_sprint.py --top-n 12

Prior evidence runs (green)

Notes

  • this PR keeps the visual validation uniqueness gates introduced earlier and extends checkout hardening to all remaining workflow entry points.

@iahncajigas
Copy link
Copy Markdown
Contributor Author

Fresh CI evidence on head :\n- validation-pdf: https://github.com/cajigaslab/nSTAT-python/actions/runs/22620617812 (success)\n- full-parity-nightly: https://github.com/cajigaslab/nSTAT-python/actions/runs/22620618645 (success)

@iahncajigas
Copy link
Copy Markdown
Contributor Author

Fresh CI evidence on branch head b0e3a1d:

@iahncajigas
Copy link
Copy Markdown
Contributor Author

Closing: superseded by v0.3.0 merge (PR #50) and subsequent work. The codex-generated fixtures, CI workflows, and demo examples were replaced by hand-curated implementations. All 484 Matlab methods are now verified in PORTING_MAP.md.

@iahncajigas iahncajigas deleted the codex/parity-line-review-20260303 branch March 11, 2026 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant